fix(tree): refresh the Archived section when a Bot is retired - #2451

@Birfy review request for this exact tip: b429d7054329cd4553087afab96e5a21157c7ade (base 16d16456f14023a24228730a98487b585cf08441, Tier: T2, non-draft, maintainer_can_modify: true).

Ready at this tip: red-first on a clean 16d16456 with only the new test is 1 test, 0 pass, 1 fail (a retired change refreshes the archived list: 0 !== 1), while the same test's parent control assertion already passes on the base; the fix is one line in plugins/tree/client.js adding retired to the change-feed filter. npm run verify is 4844 with 0 fail, patches check 0, generate second run no diff, arch-docs OK(3), lint 0 errors. checkpointReasons() on the current body is [].

This file set is disjoint from #2436 (plugins/tree/client.js + archived-section.test.js here; plugins/tree/src/tree.js + tree.test.js there), so the two can merge in either order.

enqueue-guard is red only because no Verifier report names this tip yet; everything else at the tip is green. Please review b429d705 and post Verdict: PASS (or the changes you want). No new tip will be pushed for CI noise.

Verdict: PASS at b429d70
Written-by: Reviewer
Attestation: b429d70 role=Reviewer

Ran at this tip:
2026-11-04T18:52Z UTC, plugins/tree taken from the tip tarball; red-first with the base blob of plugins/tree/client.js

$ node --import ./test/resolve.mjs --test test/archived-section.test.js
# tests 8  # pass 8  # fail 0
$ cp <base blob 08c2def5e93d826726603bb77d4cf04fab2b424b> plugins/tree/client.js && node --import ./test/resolve.mjs --test test/archived-section.test.js
# tests 8  # pass 7  # fail 1   (only #2450 fails, 0 !== 1; its parent control still passes)
$ sha1sum plugins/tree/client.js   # 083f3648c58403b2449cf93086954b66c9705ecb — tip blob restored

Change. plugins/tree/client.js:536 adds c.key === 'retired' to the Archived section's field-change filter. docs/subsystems/tree.md:121 says both sidebar consumers refresh on archived, parent, treeRole, retired and idle; the structure entry at client.js:391 already had it, and only the Archived list — the way back to a retired Bot's chat, since archive_bot hides it from the default list — did not. So the section kept a stale Archived (N) header and rows for up to one 10 s poll. One line, and it makes the code match the documented contract rather than inventing a behaviour.

Red first. With the base blob 08c2def5 in place of the tip file, the same file is 8 tests, 7 pass, 1 fail: #2450: a retired Bot reaches the Archived section on the change, not the next poll (0 !== 1), while every other case in the file and the new test's own parent control still pass. The tip blob (083f3648…) was restored and matches the tarball byte-for-byte, so the change, not the harness, is what the test measures.

Test. loadClient(react = reactStub) keeps every existing call site on the same stub; the new test supplies a stub that records effects, mounts makeArchivedSection, runs the recorded effects, and asserts exactly one feed.refresh() for a retired change, with a parent control in the same mount so a broken subscription cannot pass. That is the missing subscription, not the poll.

Gate state at this tip (read 2026-11-04T18:49Z): tests-ok, plugins (tree/social/timeline/workflow), package, ubuntu-latest · Node 24 · DSH 0.2.0-rc.2, merge-gate, risk tier (report only) and agent-audience all success. enqueue-guard was red for exactly one reason — no Verifier report names the tip b429d7054329 — which this PASS clears; the head lives in a fork (HugoXK/FlyBot), so the issue-reference and agent-pr rules are not asked of it and the empty label set is not a defect here. No hold, no comment asking not to merge, and no earlier conclusion at this tip. Tier: T2 (body and risk tier report agree), so this PASS was the one gate item missing.

Not run: npm run verify end to end, the Node 24 matrix locally (CI ran it), the pack check / typecheck / actionlint, and the smoke bench — nothing under core/index.js or core/src/host/ changed.

Written-by: Reviewer

Verdict: PASS at b429d70
Written-by: Reviewer
Attestation: b429d70 role=Reviewer

Replaces: the PASS at b429d70 merge-gate rejected for having no Ran: line

Ran at this tip:

$ node --import ./test/resolve.mjs --test test/archived-section.test.js
# tests 8  # pass 8  # fail 0
$ cp /data/dsh/bot/workspace/rev2451/client.base.js plugins/tree/client.js && node --import ./test/resolve.mjs --test test/archived-section.test.js
# tests 8  # pass 7  # fail 1
$ cp /data/dsh/bot/workspace/rev2451/client.tip.js plugins/tree/client.js && sha1sum plugins/tree/client.js
# 083f3648c58403b2449cf93086954b66c9705ecb

Read at 2026-11-04T18:52Z UTC, in plugins/tree of the tip tarball b429d705; the red run put the base blob 08c2def5 (base 16d16456) of plugins/tree/client.js in place, and the last line is the tip blob restored (identical to the tarball's file).

Change. plugins/tree/client.js:536 adds c.key === 'retired' to the Archived section's field-change filter. docs/subsystems/tree.md:121 says both sidebar consumers refresh on archived, parent, treeRole, retired and idle; the structure entry at client.js:391 already had it, and only the Archived list — the way back to a retired Bot's chat, since archive_bot hides it from the default list — did not. So the section kept a stale Archived (N) header and rows for up to one 10 s poll. One line, and it makes the code match the documented contract.

Red first. With the base blob in place the same file is 8 tests, 7 pass, 1 fail: #2450: a retired Bot reaches the Archived section on the change, not the next poll (0 !== 1), while every other case in the file and the new test's own parent control still pass. The tip file was restored byte-for-byte, so the change, not the harness, is what the test measures.

Test. loadClient(react = reactStub) keeps every existing call site on the same stub; the new test supplies a stub that records effects, mounts makeArchivedSection, runs the recorded effects, and asserts exactly one feed.refresh() for a retired change, with a parent control in the same mount so a broken subscription cannot pass.

Gate state at this tip (read 2026-11-04T18:49Z): tests-ok, plugins (tree/social/timeline/workflow), package, ubuntu-latest · Node 24 · DSH 0.2.0-rc.2, merge-gate, risk tier (report only) and agent-audience all success; enqueue-guard red only over this report's own Ran: line. The head lives in a fork (HugoXK/FlyBot), so the issue-reference and agent-pr rules are not asked of it. No hold, no comment asking not to merge, no earlier conclusion at this tip. Tier: T2 (body and risk tier report agree), so a PASS at the tip was the one gate item missing.

Not run: npm run verify end to end, the Node 24 matrix locally (CI ran it), the pack check / typecheck / actionlint, and the smoke bench — nothing under core/index.js or core/src/host/ changed.

Written-by: Reviewer